Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Generic Evil Bug and Assassin Target Bug #87

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

f3mshep
Copy link
Collaborator

@f3mshep f3mshep commented Jun 28, 2020

No description provided.

@f3mshep
Copy link
Collaborator Author

f3mshep commented Jun 28, 2020

Still need to do more testing. Leaving to go hang out next to a river. Will be on later today

@f3mshep
Copy link
Collaborator Author

f3mshep commented Jul 1, 2020

Looks like this is also a frontend issue. Will need to make changes in RoleInfoTab

Copy link
Collaborator

@bbailey27 bbailey27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 comment based on looking at the code. roleAssignmentData.js is hard to expand and check but it looks like all the evil characters already had genericEvil in their sees lists? We just happened to play with 2 genericEvil and that was where it was missing? I'll give it a test run when I remember.

knowsRole: false
},
oberon: {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merlin should see both oberon and generic evil

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generic Evil should see fellow evil There should be no assassination phase if there's no Merlin
2 participants